wayland: Add wayland-specific method to retrieve a device node path

This will be useful at least for g-c-c, in order to match libwacom
data with GdkDevices.

https://bugzilla.gnome.org/show_bug.cgi?id=770026

gdk: Pass hardware ID on gdk_device_tool_new()

And implement this on wayland, where this information is already obtained.

https://bugzilla.gnome.org/show_bug.cgi?id=770026

gdk: Add a getter for the hardware id of a GdkDeviceTool

Although scarcely used, this information may be useful to retrieve
from the windowing systems that offer this information.

https://bugzilla.gnome.org/show_bug.cgi?id=770026

gtkbindings: Add an example for gtk_binding_entry_add_signal()

Otherwise the way the varargs are supposed to be used is completely
opaque.

https://bugzilla.gnome.org/show_bug.cgi?id=770236

Update Catalan translation

Updated Polish translation

gtkbindings: Clarify that widgets need has-focus for bindings to work

https://bugzilla.gnome.org/show_bug.cgi?id=770242

Adwaita: increase border radius on button.circular...

...to allow huge circular buttons.
See https://bugzilla.gnome.org/show_bug.cgi?id=770166

quartz: Fix typo that broke debug builds

Updated French translation

Updated Brazilian Portuguese translation

Updated German translation

Updated German translation

popover: Clarify transitions-enabled deprecation

https://bugzilla.gnome.org/show_bug.cgi?id=769706

Updated Hungarian translation

stylepropertyimpl: Remove double assignment

gdk: Fix gdk_device_tool_get_serial() return value

This is a guint64, not just a guint.

https://bugzilla.gnome.org/show_bug.cgi?id=770026

gdk: Fix GdkDevice::tool-changed signal marshaller

GdkDeviceTool is an object, not a boxed type.

https://bugzilla.gnome.org/show_bug.cgi?id=770026

Update GTK+ Windows icon (now scles up to 256x256)

Also add the SVG file that was used to produce it (derived
from the old raster logo).

https://bugzilla.gnome.org/show_bug.cgi?id=768081

wayland: Use keyboard serial for implicit grab

An xdg-popup requires a serial that the compositor will compare against
its own serial and will dismiss the popup if it doesn't match.

gtk+ uses either a pointer or touch serial for its helper function
_gdk_wayland_seat_get_last_implicit_grab_serial() but if the menu is
triggered before the user has had any pointer or touch interaction with
the client, using a keyboard shortcut, there is neither pointer nor
touch serial available, and gtk+ will use 0 as the default.

As a result, the compositor will instantly dismiss the xdg-popup. In
this case, gtk+ should use the keyboard serial instead.

Track keyboard serial as well and use the keyboard serial as the value
if there is no newer pointer or touch serial available.

https://bugzilla.gnome.org/show_bug.cgi?id=768017

Minor doc cleanup

gtk-doc is smart about plural links, nowadays.

Remove an outdated comment

It described as TODO what the code right below it already does.

Updated Hungarian translation

Updated German translation

Update POTFILES.skip

wayland: Postpone processing move_to_rect params until showing

At the time of move_to_rect() is called, not all state may have been set
up on the impl gdk window, causing the position to sometimes be
slightly offset due to drap shadow margins. For now, work around this
by postponing the processing of the move_to_rect() parameters until
showing, when its more likely that all state (such as shadow margin)
has been set correctly.

https://bugzilla.gnome.org/show_bug.cgi?id=769402

gdkwindow: Use toplevel for getting root cords in move_to_rect()

The Wayland backend manages a set of fake root coordinate spaces, where
each non-relative positioned toplevel (i.e. not popups, popovers,
tooltips etc) make up the basis of separate fake root coordinate spaces.

This means that the Wayland backend doesn't have the abilitiy get a
proper root coordinate when querying on a non-toplevel GdkWindow. To
avoid this issue, first find the toplevel, while translating the anchor
rect coordinates so that they are in the toplevel window coordinate
space. Then use this toplevel to translate the coordinates to root
window coordinate space.

https://bugzilla.gnome.org/show_bug.cgi?id=769402

wayland: Don't traverse transient-ofs when faking root coordinate space

The position of each transient-of will be in fake-root coordinate
space; thus we should not accumulate all the positions making it an
offset; each window is already in fake root coordinate space.

https://bugzilla.gnome.org/show_bug.cgi?id=769402

wayland: Use effective toplevel as popup parent

When using the set transient-for as a popup parent, fetch the effective
toplevel instead, otherwise we will position against the wrong
coordinate.

https://bugzilla.gnome.org/show_bug.cgi?id=769402

Use gtk_popover_popdown/popup where appropriate

https://bugzilla.gnome.org/show_bug.cgi?id=769706

GtkPopover: Deprecate transitions-enabled

The effect of transitions-enabled=true can now be
achieved using gtk_popover_popup/popdown and the effect
of transitions-enabled=false can be achieved using
gtk_widget_show/hide.

https://bugzilla.gnome.org/show_bug.cgi?id=769706

GtkPopover: Add gtk_popover_popdown/popup

Since not chaining up in gtk_widget_show/gtk_widget_hide is not allowed,
we can't just implicitly delay the hiding in GtkPopover's hide
implementation. Fix this by introducing gtk_popover_popup() and
gtk_popover_popdown() to show or hide a popover with transition and
revert GtkPopover's show/hide implementation to apply their effect
without the transition.

https://bugzilla.gnome.org/show_bug.cgi?id=769706

Updated Indonesian translation

Updated Indonesian translation

mir: group DND, tooltips, and notifications with menu-type windows

Remove outdated comments

https://bugzilla.gnome.org/show_bug.cgi?id=768138

Use Menus to implement tooltips

The order in which tooltips are created, drawn, shown and then positioned,
always requires repositioning the surface. The tooltip window type only has
limited capability to do so. An alternative could be to use bufferstreams.

https://bugzilla.gnome.org/show_bug.cgi?id=768138

Fix execution of dialog

When a dialog is created, the mir event source is already executed on the
call stack. So without the recurse flag it will not be run in the main loop
used for the dialog.

https://bugzilla.gnome.org/show_bug.cgi?id=768138

file chooser: align date formatting with nautilus

We were treating 'yesterday' not quite the same way.

https://bugzilla.gnome.org/show_bug.cgi?id=769568

docs: Fix a dangling link

https://bugzilla.gnome.org/show_bug.cgi?id=769601

Really fix unlinkage for shm_open

Reported in https://bugzilla.gnome.org/show_bug.cgi?id=769603

Only apply type hint if it would map to a different mir surface type

Signed-off-by: Andreas Pokorny <andreas.pokorny@canonical.com>

Apply geometry hints to mir surface

Signed-off-by: Andreas Pokorny <andreas.pokorny@canonical.com>

Apply and forward title changes of gdk windows

Signed-off-by: Andreas Pokorny <andreas.pokorny@canonical.com>

Only update surface spec when there is no spec change pending

Rework window construction - only recreate surface when necessary

Signed-off-by: Andreas Pokorny <andreas.pokorny@canonical.com>

Forward repeated key presses as further down keys

Signed-off-by: Andreas Pokorny <andreas.pokorny@canonical.com>

Fix warning on newer version so mir 0.22 and newer

Signed-off-by: Andreas Pokorny <andreas.pokorny@canonical.com>

Fix gcc warning on potentially uninitialized gdk_event.

Use the surface output event to keep track of the suggested scale value

The event will be sent when the surface becomes visible on an output. With this change the GdkMirWindowImpl keeps track of the scale value and sends a configure event on change.

Signed-off-by: Andreas Pokorny <andreas.pokorny@canonical.com>

widget-factory: Don't initially show popover

It'll  be set as the popover of a GtkMenuButton later anyway which will
hide it again.

popover: Make some public API NULL-safe

Instad of potentially dereferencing a NULL-pointer, use the NULL-safe
gtk_popover_get_instance_private before the g_return_if_fail
precondition check.

label: Remove some useless casts

These were casts of a const GdkEvent* to GdkEvent* even though the
function they are passed to takes a const GdkEvent*.

Use g_snprintf instead of snprintf.

snprintf is not compatible with msvc.

git.mk: Add config.rpath to the list of ignored files

This finally cleans up the `git status` output.

Remove config.rpath from EXTRA_DIST

The config.rpath script was added by running gettextize, but it's not
really needed, as config.rpath is generated by autotools and it's
included in the dist tarball automatically.

Upstream gettextize has been fixed so it does not happen any more:
http://savannah.gnu.org/bugs/index.php?48729

Updated Czech translation

Updated Slovak translation

wayland: Link against librt

This is required for shm_open.

https://bugzilla.gnome.org/show_bug.cgi?id=769603

headerbar: Fix size allocation

The GtkHeaderBar gadget implementation was subtly broken: it called
gtk_widget_set_allocation both in gtk_header_bar_size_allocate (with
the actual allocation) and in gtk_header_bar_allocate_contents (with
the content allocation of the main gadget). Dropping the second call
fixes the render node conversion for GtkHeaderBar.

Updated Hebrew translation

Updated Hebrew translation

gdkwindow: Fix potential NULL pointer dereferences in event code

The event code could potentially dereference pointer_info if the
invariant that ENTER_NOTIFY and LEAVE_NOTIFY events are only emitted on
devices which have pointers is violated elsewhere.

Found with scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkicontheme: Don’t emit warning about fallback theme when in unit tests

In order to eliminate g_test_expect_message() (which doesn’t work with
G_LOG_USE_STRUCTURED), make the warning about the fallback theme not
existing be conditional on the icon theme search path containing a
system path. Any application code which modifies the search path does so
through appends and prepends, so this should not affect whether the
warning is emitted in production.

https://bugzilla.gnome.org/show_bug.cgi?id=769485

tests: Avoid g_test_expect_message

Instead of using g_test_expect_message, just ignore the warning.

GtkMenuButton: use :toggled instead of :clicked

:toggled is triggered on :clicked, so using :toggled lead to the menu
to be popped up at the same time, while allowing to use the toggle state
and avoiding any need to a hack to prevent recursion, which somehow
wasn't enough for double emission of GtkMenuToolButton:show-popup.

https://bugzilla.gnome.org/show_bug.cgi?id=769287

Updated Polish translation

csslookup: Remove useless cast

The style passed to _gtk_css_lookup_resolve is already a
GtkCssStaticStyle.

Updated German translation
(cherry picked from commit 070c4f74868c31cc9316bbad38e2ad42adc43443)

W32: Prefer the deadkey combinations that the OS uses

Pick the W32 API for possible deadkey+<something> combinations
and prefer these to other sources of deadkey combos.
Specifically, if W32 API supports at least one combo for a particular
deadkey, only use that data and do not attempt to do other, unsupported
combinations, even if they make sense otherwise.
This is needed to, for example, correctly support US-International
keyboard layout, which produces a combined character for <' + a>
combo, but not for <' + s>, for example.

This is achieved by stashing all the deadkeys that we find in
an array, then doing extra loop through all virtual key codes and
trying to combine them with each of these deadkeys. Any combinations
that produce a single character are cached for later use.

In GTK Simple IM context, call a new GDK W32 function to do a lookup
on that cached combination table early on, among the "special cases"
(which are now partially obsolete).

A limitation of this code is that combinations with more than
one deadkey are not supported, except for combinations that consist
entirely of 2 known deadkeys. The upshot is that lookups should
be relatively fast, as deadkey array stays small and the combination
tree stays shallow.

Note that the use of ToUnicodeEx() seems suboptimal, as it should
be possible to just load a keyboard library (KBD*.DLL) manually
and obtain and use its key table directly. However, that is much more
complicated and would result in a significant rewrite of gdkkeys-win32.
The code from this commit, though hacky, is a direct addition to
existing code and should cover vast majority of the use-cases.

https://bugzilla.gnome.org/show_bug.cgi?id=569581

GDK W32: Cache multiple keyboard layouts simultaneously

This changes the group/level semantic.
Previously W32 backend used "group 0/1" to denote "AltGr OFF/ON"
and "level 0/1" to denote "Shift is OFF/ON".
Now "group" means "keyboard layout" and there can be up to 255 groups,
while AltGr and Shift are combined into a single level enum that
takes values between 0 and 4.
Unlike X, W32 doesn't do effective group overriding, meaning that
it will never tell the caller that a different group was actually
used (even for universal keys, such as Enter), because key symbol
table is completely fabricated and there's no point in trying to
save a few of kilobytes of RAM by not duplicating universal key
records for all groups.

Also contains many whitespace changes (tab elimination, fixed
indentation) and cleanup (axed a few global variables, these are
now accessed via the default keymap).

https://bugzilla.gnome.org/show_bug.cgi?id=768722

shortcutlabel: Make parameter names match

widget: remove some unneeded function prototypes

filechooserwidget: Remove unused enum

label: Destroy the popup menu on unmap

Updated Spanish translation

Modify git.mk to handle our split NLS directories

GTK+ has two directories for translations: the default 'po' and the
additional 'po-properties' for the GObject properties translation
domain.

Since the content of the translations directories are filled by
autoreconf and gettext, and are duplicated between the two gettext
domains we use, we should handle this ad hoc inside git.mk, instead of
trying to catch up by adding po-properties files in GITIGNOREFILES.

Add ignore file for macro dir

The contents of the macro directory are too complicated for git.mk to
handle: the contents are filled by autotools and may change between
systems or autotools releases; and we additionally ship our own m4
macros.

To avoid unnecessary noise on the `git status` output we should simply
add an ignore file specifically for the macro directory.

Skip GtkShortcutLabel::accelerator in notify test

This property is not freely settable, since the string
gets parsed.

menu: Fix up new properties

The new positioning-related properties had some quality of
implementation issues, such as incorrect initial values and
excessive change notification. This broke the notify test.

Make icontheme test work with structured logging

Since we've opted in to G_LOG_USE_STRUCTURED, this test
now needs to do its thing using a log writer function.

Skip GdkDrawingContext in default value tests

GdkDrawingContext objects can't be freely constructed.

Require a new version of GLib

We need GLib 2.49.4 because of the new API requirements introduced by
commit 936c1649.

https://bugzilla.gnome.org/show_bug.cgi?id=769451

Updated Spanish translation

Updated Spanish translation

Update the docs for gtk_window_get_position()

Drop mentions of GnomeClient, and add mentions of different windowing
systems instead of just assuming that we're using X11.

Add a release not about GtkTable expand flags

gtkbox: remove unnecessary queue_compute_expand()

The expand child property does not have any effect on the
expand state of the GtkBox, so queuing a compute_expand
when changing it is not needed.

https://bugzilla.gnome.org/show_bug.cgi?id=769162

gtktable: don't try to propagate expand related child props in compute_expand()

It tried to set the expand state if either xexpand/yexpand where true.
Due to a missing queue_compute_expand when adding a child it actually
only computed the expand state in case a child queued after being added
or in case a child had the expand property set (see optimization in
gtk_widget_set_parent)

In my case this broke layouts as a child of GtkCombBox started setting
an exand flag with 3.20 which queued a compute_expand, which in turn
propagated an expand child props set for a cell in the same table up
and overrode the expand child prop of a parent GtkBox.

This removes the custom compute_expand implementation to match the
behaviour of GtkBox (don't propagate child prop expand flags
but let child expand flags override the child props) and not get random
expand behaviour depending on whether and when child widgets set their
expand state.

https://bugzilla.gnome.org/show_bug.cgi?id=769162

Updated Lithuanian translation

demos: Fix build failure

gtk+/demos/gtk-demo/css_blendmodes.c: In function ‘update_css_for_blend_mode’:
gtk+/demos/gtk-demo/css_blendmodes.c:49:26: error: format not a string literal, argument types not checked [-Werror=format-nonliteral]
                          blend_mode);
                          ^~~~~~~~~~

https://bugzilla.gnome.org/show_bug.cgi?id=769236

docs: trivial build fix

The error was:
gtk3-scan.c:193: undefined reference to `gtk_shortcuts_label_get_type'
collect2: error: ld returned 1 exit status

It's since commit 7543cd8ce419a4d660f0ff7614f0c6e1cac81804, which made
the GtkShortcutLabel class public.

Port GtkPrintOperationPortal to gtk_window_export_handle

Port gtk_show_uri_on_window to gtk_window_export_handle

As part of this, make it use the new
g_app_info_launch_default_for_uri_async, since we need a way
to unexport the window handle afterwards.

Port GtkFileChooserNativePortal to gtk_window_export_handle

Put window exporting behind a display protocol agnostic API

Introduce a private API meant for abstracting how to get a handle
of a window that can be shared with other processes. The API is
async, since some implementations will require that. Currently,
only X11 is supported, which doesn't.

Based on a patch by Jonas Adahl.

GDK W32: Support UTF-16 surrogate pairs passed via VK_PACKET

This is, essentially, a piece of g_utf16_to_ucs4() built into GDK
W32 keyboard message processing.

https://bugzilla.gnome.org/show_bug.cgi?id=769126

css: Fix case where we didn't convert pt => px

Fixes gtk_widget_override_font() not behaving properly since commit
df08fc91bdc1d2e4c866122304fabe4dd298a7de.

https://bugzilla.gnome.org/show_bug.cgi?id=768902

css: Rename functions

I don't want to use a generic function for font sizes as font sizes are
special. Plus, the function is only used in one place.

Properly declare the font CSS property

https://bugzilla.gnome.org/show_bug.cgi?id=769004